Skip to content

ext/session:Fix session handler missing create_sid() diagnostic level - #23549

Closed
arshidkv12 wants to merge 2 commits into
php:masterfrom
arshidkv12:session-deprecated
Closed

ext/session:Fix session handler missing create_sid() diagnostic level#23549
arshidkv12 wants to merge 2 commits into
php:masterfrom
arshidkv12:session-deprecated

Conversation

@arshidkv12

Copy link
Copy Markdown
Contributor

@NickSdot

NickSdot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

ref: #23298

@arshidkv12

Copy link
Copy Markdown
Contributor Author

Thank you

@arshidkv12 arshidkv12 closed this Sep 3, 2026
@NickSdot

NickSdot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Thank you

I just meant to add context; no need to close I guess.

@arshidkv12 arshidkv12 reopened this Sep 3, 2026
@Girgias

Girgias commented Sep 5, 2026

Copy link
Copy Markdown
Member

The RFC stipulates a warning:

And the following warning will be emitted when implementing SessionHandlerInterface on a class which doesn't define the create_sid() or validateId() methods:

@Girgias Girgias closed this Sep 5, 2026
@arshidkv12
arshidkv12 deleted the session-deprecated branch September 5, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants